Skip to content

Added the test case for Port Channel cleanup into runtest.sh#157

Merged
abdosi merged 2 commits intoAzure:masterfrom
abdosi:pc_cleanup
Sep 9, 2020
Merged

Added the test case for Port Channel cleanup into runtest.sh#157
abdosi merged 2 commits intoAzure:masterfrom
abdosi:pc_cleanup

Conversation

@abdosi
Copy link
Contributor

@abdosi abdosi commented Sep 8, 2020

Test make sure cleanup happens of Port-channel Kernel devices.
This test case track the fixes done by PR:
sonic-net/sonic-swss#1407
sonic-net/sonic-swss#1159

Verified on 201911 KVM Image test case is passed

johnar@6b57928a7072:/data/Networking-acs-sonic-mgmt/tests$ !252
sudo py.test --inventory "veos.vtb" --host-pattern vlab-01 --module-path "../ansible/library/" --testbed vms-kvm-t0 --testbed_file "vtestbed.csv" -vvv pc/test_po_cleanup.py --skip_sanity
=============================================================================================== test session starts ===============================================================================================
platform linux2 -- Python 2.7.12, pytest-4.6.5, py-1.8.1, pluggy-0.13.1 -- /usr/bin/python
cachedir: .pytest_cache
ansible: 2.8.7
rootdir: /data/Networking-acs-sonic-mgmt/tests, inifile: pytest.ini
plugins: ansible-2.2.2, xdist-1.28.0, repeat-0.8.0, forked-1.1.3
collected 1 item

pc/test_po_cleanup.py::test_po_cleanup [[APASSED [100%]

====================================== 1 passed in 174.22 seconds =======================================

Updated on master with result:

Platform: x86_64-kvm_x86_64-r0
HwSKU: Force10-S6000
ASIC: vs
Serial Number: 000000
Uptime: 00:22:06 up 5 min, 1 user, load average: 0.35, 0.46, 0.22

Docker images:
REPOSITORY TAG IMAGE ID SIZE
docker-syncd-vs latest 3d9ea8ec98b0 362MB
docker-syncd-vs master.476-8d285b46 3d9ea8ec98b0 362MB
docker-teamd latest 39d361a54835 386MB
docker-teamd master.476-8d285b46 39d361a54835 386MB
docker-nat latest c4f0345fe40c 389MB
docker-nat master.476-8d285b46 c4f0345fe40c 389MB

johnar@6b57928a7072:/data/Networking-acs-sonic-mgmt/tests$ sudo py.test --inventory "veos.vtb" --host-pattern vlab-01 --module-path "../ansible/library/" --testbed vms-kvm-t0 --testbed_file "vtestbed.csv" -vvv
pc/test_po_cleanup.py --skip_sanity
========================================== test session starts ==========================================
platform linux2 -- Python 2.7.12, pytest-4.6.5, py-1.8.1, pluggy-0.13.1 -- /usr/bin/python
cachedir: .pytest_cache
ansible: 2.8.7
rootdir: /data/Networking-acs-sonic-mgmt/tests, inifile: pytest.ini
plugins: ansible-2.2.2, xdist-1.28.0, repeat-0.8.0, forked-1.1.3
collected 1 item

pc/test_po_cleanup.py::test_po_cleanup ^A^[[APASSED [100%]

Signed-off-by: Abhishek Dosi [email protected]

Test make sure cleanup happens of Port-channel Kernel devices.
This test case track the fixes done by PR:
sonic-net/sonic-swss#1407
sonic-net/sonic-swss#1159

Signed-off-by: Abhishek Dosi <[email protected]>
@abdosi abdosi requested review from daall and lguohan September 8, 2020 20:36
tacacs/test_ro_user.py \
telemetry/test_telemetry.py"
telemetry/test_telemetry.py \
pc/test_po_cleanup.py"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you keep the list in alphabetical order, please?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@daall : Updated

@daall
Copy link
Contributor

daall commented Sep 9, 2020

Verified on 201911 KVM Image test case is passed

Did you check with the master image as well?

Signed-off-by: Abhishek Dosi <[email protected]>
@abdosi
Copy link
Contributor Author

abdosi commented Sep 9, 2020

Verified on 201911 KVM Image test case is passed

Did you check with the master image as well?

Updated with master result also in description.

@abdosi abdosi merged commit a88735c into Azure:master Sep 9, 2020
@abdosi abdosi deleted the pc_cleanup branch September 9, 2020 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants